Skip to content

fix: treat LinkedEvents 404/410 as ObjectDoesNotExistError#459

Merged
mhieta merged 1 commit into
mainfrom
fix/sentry-httperror-404-410
Jan 23, 2026
Merged

fix: treat LinkedEvents 404/410 as ObjectDoesNotExistError#459
mhieta merged 1 commit into
mainfrom
fix/sentry-httperror-404-410

Conversation

@mhieta

@mhieta mhieta commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

Event/place resolvers now use shared LinkedEvents fetch helper.

404 and 410 are converted to ObjectDoesNotExistError so they are not reported to Sentry as HTTPError.

Refs: PT-1990

@azure-pipelines

Copy link
Copy Markdown

KULTUS-API branch is deployed to platta: https://kultus-pr459.api.dev.hel.ninja 🚀🚀🚀

@mhieta mhieta force-pushed the fix/sentry-httperror-404-410 branch from 0794e83 to 552de4a Compare January 23, 2026 11:38
@azure-pipelines

Copy link
Copy Markdown

KULTUS-API branch is deployed to platta: https://kultus-pr459.api.dev.hel.ninja 🚀🚀🚀

@mhieta mhieta marked this pull request as ready for review January 23, 2026 12:03
@mhieta mhieta requested a review from a team as a code owner January 23, 2026 12:03
@azure-pipelines

Copy link
Copy Markdown

KULTUS-API branch is deployed to platta: https://kultus-pr459.api.dev.hel.ninja 🚀🚀🚀

Event/place resolvers now use shared LinkedEvents fetch helper.

404 and 410 are converted to ObjectDoesNotExistError so they are not
reported to Sentry as HTTPError.

Refs: PT-1990
@mhieta mhieta force-pushed the fix/sentry-httperror-404-410 branch from 552de4a to cca3f5f Compare January 23, 2026 12:24
@sonarqubecloud

Copy link
Copy Markdown

@azure-pipelines

Copy link
Copy Markdown

KULTUS-API branch is deployed to platta: https://kultus-pr459.api.dev.hel.ninja 🚀🚀🚀

@mhieta mhieta merged commit 9724b21 into main Jan 23, 2026
26 checks passed
@mhieta mhieta deleted the fix/sentry-httperror-404-410 branch January 23, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants